Skip to content

CQL Test Server for CQL Tests Runner#360

Open
cmoesel wants to merge 4 commits intomasterfrom
test-server
Open

CQL Test Server for CQL Tests Runner#360
cmoesel wants to merge 4 commits intomasterfrom
test-server

Conversation

@cmoesel
Copy link
Copy Markdown
Member

@cmoesel cmoesel commented Apr 16, 2026

This PR implements a test server that exposes a $cql target to allow the CQL Tests Runner to run tests against the cql-execution engine. Rather than repeat all the details here, I'll refer you to the test-server/README.md file in this branch.

I initially developed this as a standalone project outside of cql-execution that pulled in cql-execution as an NPM dependency. This PR integrates it into the cql-execution project as a sub-project with its own package.json and configuration files. In addition to general review of the code, I'm also interested in thoughts on this approach. Should we:

  • Keep it as I have it here (a sub-project)?
  • Keep it in cql-execution but try to integrate it more closely (i.e., share package.json and configuration files)?
  • Or introduce it as a completely separate project outside of cql-execution (and if so, where)?

I have not yet invested in updating GitHub actions to automatically run the tests or other checks in the sub-project. I will do that after we have confirmed the approach.

NOTE: The npm audit check is currently failing due to a reported vulnerability in elliptic. There is currently no fix for that.

Submitter:

  • This pull request describes why these changes were made
  • Code diff has been done and been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
  • Tests are included and test edge cases
  • Tests have been run locally and pass
  • Code coverage has not gone down and all code touched or added is covered.
  • Code passes lint and prettier (hint: use npm run test:plus to run tests, lint, and prettier)
  • All dependent libraries are appropriately updated or have a corresponding PR related to this change
  • [n/a] cql4browsers.js built with npm run build:browserify if source changed.

Reviewer:

Name:

  • Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
  • The tests appropriately test the new code, including edge cases
  • You have tried to break the code

cmoesel added 3 commits April 16, 2026 11:12
- fully self-contained project as subfolder
- uses release version of cql-execution
Ideally we would use the code from src directly, but typescript isn't happy
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.58%. Comparing base (041ed96) to head (5472310).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #360   +/-   ##
=======================================
  Coverage   87.58%   87.58%           
=======================================
  Files          52       52           
  Lines        4607     4607           
  Branches     1297     1297           
=======================================
  Hits         4035     4035           
  Misses        359      359           
  Partials      213      213           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants